Skip to content

chore: scope changelog + bugbot to breaking changes only#2191

Merged
mikasenghaas merged 1 commit intomainfrom
chore/changelog-breaking-only
Apr 6, 2026
Merged

chore: scope changelog + bugbot to breaking changes only#2191
mikasenghaas merged 1 commit intomainfrom
chore/changelog-breaking-only

Conversation

@mikasenghaas
Copy link
Copy Markdown
Member

@mikasenghaas mikasenghaas commented Apr 3, 2026

Summary

  • Narrows CHANGELOG.md scope to breaking config changes only (renames, removals, moves) — not additive features or default tweaks
  • Updates .cursor/BUGBOT.md rule to match, with clearer criteria and updated config paths (src/prime_rl/configs/)

🤖 Generated with Claude Code


Note

Low Risk
Low risk documentation-only change that adjusts contribution/enforcement guidance but does not modify runtime behavior.

Overview
Updates .cursor/BUGBOT.md to require CHANGELOG.md entries only for breaking config changes (renamed/removed/moved fields), explicitly excluding additive fields and default tweaks, and points to the current config location src/prime_rl/configs/.

Tweaks the CHANGELOG.md header/description to match this narrower breaking-changes-only scope.

Reviewed by Cursor Bugbot for commit 8367349. Bugbot is set up for automated code reviews on this repo. Configure here.

Non-breaking additions (new fields with defaults, new features) were
cluttering the changelog and creating unnecessary PR friction. Narrow
the scope to renames, removals, and moves that require users to update
existing configs.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@mikasenghaas mikasenghaas requested review from JannikSt and samsja April 3, 2026 12:10
@mikasenghaas mikasenghaas marked this pull request as ready for review April 6, 2026 15:07
@mikasenghaas mikasenghaas merged commit ecf1ae2 into main Apr 6, 2026
7 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants